Skip to content

test: use Rstest projects to define tests#12729

Merged
chenjiahan merged 4 commits intomainfrom
rstest-projects
Jan 14, 2026
Merged

test: use Rstest projects to define tests#12729
chenjiahan merged 4 commits intomainfrom
rstest-projects

Conversation

@9aoy
Copy link
Contributor

@9aoy 9aoy commented Jan 14, 2026

Summary

Use Rstest projects to define Rspack tests.

https://rstest.rs/guide/basic/projects

Related links

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

Copilot AI review requested due to automatic review settings January 14, 2026 10:00
@github-actions github-actions bot added the team The issue/pr is created by the member of Rspack. label Jan 14, 2026
@netlify
Copy link

netlify bot commented Jan 14, 2026

Deploy Preview for rspack canceled.

Name Link
🔨 Latest commit e77d3b8
🔍 Latest deploy log https://app.netlify.com/projects/rspack/deploys/69677625d8d8a400086d77aa

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request refactors the test configuration to use Rstest's projects feature, consolidating the separate hot test configuration into a single multi-project setup.

Changes:

  • Migrated from separate test configurations to a projects-based approach using defineProject and projects array
  • Consolidated rstest.config.hot.ts into the main config as a separate project named 'hottest'
  • Moved environment variables (like RUST_BACKTRACE) and test execution settings from package.json scripts into the configuration file
  • Updated @rstest/core from version 0.7.8 to 0.7.9

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
tests/rspack-test/rstest.config.ts Refactored to use projects structure with shared config and two projects (base and hottest), moved pool config to top level
tests/rspack-test/rstest.config.hot.ts Deleted file - functionality merged into main config as 'hottest' project
tests/rspack-test/package.json Simplified test scripts and updated @rstest/core dependency to 0.7.9
package.json Updated @rstest/core dependency to 0.7.9
pnpm-lock.yaml Updated lock file entries for @rstest/core version 0.7.9
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 14, 2026

Rsdoctor Bundle Diff Analysis

Found 5 projects in monorepo, 0 projects with changes.

📊 Quick Summary
Project Total Size Change
react-10k 5.7 MB 0
react-1k 825.4 KB 0
react-5k 2.7 MB 0
rome 984.3 KB 0
ui-components 2.1 MB 0

Generated by Rsdoctor GitHub Action

@github-actions
Copy link
Contributor

github-actions bot commented Jan 14, 2026

📦 Binary Size-limit

Comparing e77d3b8 to feat: support portable cache and disable by default (#12680) by jinrui

🙈 Size remains the same at 47.97MB

@9aoy 9aoy requested a review from hardfist as a code owner January 14, 2026 10:12
@codspeed-hq
Copy link

codspeed-hq bot commented Jan 14, 2026

Merging this PR will not alter performance

✅ 16 untouched benchmarks
⏩ 1 skipped benchmark1


Comparing rstest-projects (e77d3b8) with main (25020fd)

Open in CodSpeed

Footnotes

  1. 1 benchmark was skipped, so the baseline result was used instead. If it was deleted from the codebase, click here and archive it to remove it from the performance reports.

Copy link
Member

@chenjiahan chenjiahan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@chenjiahan chenjiahan changed the title test: use Rstest projects to define Rspack tests. test: use Rstest projects to define tests Jan 14, 2026
@chenjiahan chenjiahan merged commit 63a410a into main Jan 14, 2026
54 checks passed
@chenjiahan chenjiahan deleted the rstest-projects branch January 14, 2026 14:04
LingyuCoder pushed a commit that referenced this pull request Jan 16, 2026
* test: use rstest projects

* test: update

* test: output.module

* test: update
chenjiahan pushed a commit that referenced this pull request Jan 16, 2026
* test: use rstest projects

* test: update

* test: output.module

* test: update
hardfist added a commit that referenced this pull request Feb 5, 2026
* feat: treeshake share

* fix(module-federation): correct shared module name and optimize tree shaking

fix incorrect module name in shared config and test cases
change async process_module to sync using block_on for better performance
update shared exports optimization to handle side effects properly

* feat(module-federation): add treeshakeSharedExcludedPlugins option

* chore: add watchRun

* fix(shared-modules): handle undefined moduleToHandlerMapping and prevent tree-shaking

Add fallback for undefined moduleToHandlerMapping in shared container entry
Include additional plugins in filter list and prevent tree-shaking of shared modules by adding console logs when collectShared is true

* test(container-1-5): add ui-lib package for treeshake shared infer strategy

* fix(shared_used_exports): handle scoped package names in shared exports
refactor(node_binding): remove redundant exports in wasi files

* chore: update critical file

* feat(module-federation): add target and plugins to build info for treeshaking

Add target and plugins fields to build info when treeshaking is enabled in module federation. Also rename 'reshake' to 'reShake' for consistency across the codebase. These changes support better treeshaking analysis and configuration.

* docs: update field

* feat: add treeshakeSharedPlugins

* fix: lint

* fix: lockfile

* fix: lint

* chore: rename config

* chore(deps): update patch npm dependencies (#12647)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update patch crates (#12646)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore: run ci/eco-ci/eco-benchmark on v2 branch (#12650)

* fix: report error when access module_graph in loader (#12639)

* fix: report error when access module_graph in loader

* Apply suggestions from code review

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* feat(rstest): add preserveNewUrl option to keep new URL untouched (#12632)

* feat(rstest): add preserveNewUrl option to keep new URL untouched

* fix: format Rust code with rustfmt

* fix(lint): enable @typescript-eslint/await-thenable rule (#12644)

* fix(lint): enable await-thenable rule and fix violation

- Enable @typescript-eslint/await-thenable rule (off -> error)
- Remove unnecessary await on syncTraceEvent() which returns void

* Update rslint.json

---------

Co-authored-by: neverland <jait.chen@foxmail.com>

* refactor: afterCodeGeneration hook use read only compilation ref (#12655)

* fix(lint): enable @typescript-eslint/require-await rule (#12664)

fix(lint): enable require-await rule and fix violations

- Enable @typescript-eslint/require-await rule (off -> error)
- FileSystem.ts: Add missing return for Promise, remove async
- SubresourceIntegrityPlugin.ts: Use Promise.resolve() instead of async
- browser/service.ts: Remove async from function that only throws
- loader-runner/worker.ts: Remove async from .then/.catch callbacks

* chore(deps): update github-actions (#12645)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* fix(lint): enable @typescript-eslint/return-await rule (#12665)

* refactor: make compilation readonly for runtimeModule hook (#12670)

* feat: add constructor name for JS runtime modules (#12673)

feat: add constructor name for JS runtime module

* fix: incremental rebuild no code generation entry panic (#12643)

* fix: incremental rebuild bug

* add test case

* fix: ci

* refactor: unify artifact clear logic (#12677)

* refactor: use DerfOpt's take/replace assist artifact mutation (#12659)

* refactor: remove Default trait of DerefOption
* refactor: `replace` is better than `insert`

* feat: rspack_cacheable context expose project root (#12685)

* feat: rspack_cacheable context expose project root

* fix: comment

* feat: rspack_cacheable add portable path & portable string (#12688)

feat: add portable path & portable string

* fix: inlined export invalid syntax comment (#12681)

* feat: add `requireAlias` option to control require variable renaming (#12686)

* feat: add requireRename option to control require variable renaming

* Update packages/rspack/src/config/types.ts

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update tests/rspack-test/configCases/parsing/renaming-disabled/index.js

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* rename

* rename

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix(test): flaky test due to calling toJSON while hot updating (#12694)

* feat: impl `sync_trace` for `logger` tracing layer (#12687)

* Sync node events for logger tracing

* Align sync_trace

* Remove unless info

* Cargo shear

* Use one writer

* Filter JavaScript

* Handle timestamps

* Clippy

* Clippy

* feat: add warning when requireAlias is disabled (#12700)

* refactor: rename tree shaking field name

* revert: "feat: add warning when requireAlias is disabled" (#12702)

Revert "feat: add warning when requireAlias is disabled (#12700)"

This reverts commit eb5635c.

* feat: use rspack-vue-loader to support Vue hot update (#12698)

feat: use rspack-vue-loader to support hotupdate

* chore: update artificat

* chore: replace async processAssets hooks with sync versions

* fix: `environments.importMetaDirnameAndFilename` not work (#12708)

fix: `environments.importMetaDirnameAndFilename` should work

* fix: dynamic entry panic when disable build cache (#12696)

* fix(context-replacement): only apply changes when regex matches (#12657)

fix(ContextReplacementPlugin): only apply changes when regex matches

The plugin was applying transformations (recursive, regExp, critical) to
all require.context() calls regardless of whether the configured regex
pattern matched the request. This fix ensures all transformations only
apply when the regex successfully matches.

Closes #12656

* docs: add missed doc

* fix: remove ueless dep

* chore: update lock file

* fix(mf): handle missing chunk gracefully and improve expose chunk name logic (#12709)

* fix(type): runtimeModule.source (#12713)

* fix(type): runtimeModule.source

* fix: lint

* fix: lint

* chore: release 1.7.2 (#12710)

Release 0.7.2

rspack@0.7.2
rspack_allocator@0.7.2
rspack_binding_api@0.7.2
rspack_binding_build@0.7.2
rspack_binding_builder@0.7.2
rspack_binding_builder_macros@0.7.2
rspack_browser@0.7.2
rspack_browserslist@0.7.2
rspack_cacheable@0.7.2
rspack_cacheable_macros@0.7.2
rspack_collections@0.7.2
rspack_core@0.7.2
rspack_error@0.7.2
rspack_fs@0.7.2
rspack_futures@0.7.2
rspack_hash@0.7.2
rspack_hook@0.7.2
rspack_ids@0.7.2
rspack_javascript_compiler@0.7.2
rspack_loader_lightningcss@0.7.2
rspack_loader_preact_refresh@0.7.2
rspack_loader_react_refresh@0.7.2
rspack_loader_runner@0.7.2
rspack_loader_swc@0.7.2
rspack_loader_testing@0.7.2
rspack_location@0.7.2
rspack_macros@0.7.2
rspack_napi@0.7.2
rspack_napi_macros@0.7.2
rspack_paths@0.7.2
rspack_plugin_asset@0.7.2
rspack_plugin_banner@0.7.2
rspack_plugin_case_sensitive@0.7.2
rspack_plugin_circular_dependencies@0.7.2
rspack_plugin_copy@0.7.2
rspack_plugin_css@0.7.2
rspack_plugin_css_chunking@0.7.2
rspack_plugin_devtool@0.7.2
rspack_plugin_dll@0.7.2
rspack_plugin_dynamic_entry@0.7.2
rspack_plugin_ensure_chunk_conditions@0.7.2
rspack_plugin_entry@0.7.2
rspack_plugin_esm_library@0.7.2
rspack_plugin_externals@0.7.2
rspack_plugin_extract_css@0.7.2
rspack_plugin_hmr@0.7.2
rspack_plugin_html@0.7.2
rspack_plugin_ignore@0.7.2
rspack_plugin_javascript@0.7.2
rspack_plugin_json@0.7.2
rspack_plugin_lazy_compilation@0.7.2
rspack_plugin_library@0.7.2
rspack_plugin_lightning_css_minimizer@0.7.2
rspack_plugin_limit_chunk_count@0.7.2
rspack_plugin_merge_duplicate_chunks@0.7.2
rspack_plugin_mf@0.7.2
rspack_plugin_module_info_header@0.7.2
rspack_plugin_module_replacement@0.7.2
rspack_plugin_no_emit_on_errors@0.7.2
rspack_plugin_progress@0.7.2
rspack_plugin_real_content_hash@0.7.2
rspack_plugin_remove_duplicate_modules@0.7.2
rspack_plugin_remove_empty_chunks@0.7.2
rspack_plugin_rsdoctor@0.7.2
rspack_plugin_rslib@0.7.2
rspack_plugin_rstest@0.7.2
rspack_plugin_runtime@0.7.2
rspack_plugin_runtime_chunk@0.7.2
rspack_plugin_schemes@0.7.2
rspack_plugin_size_limits@0.7.2
rspack_plugin_split_chunks@0.7.2
rspack_plugin_sri@0.7.2
rspack_plugin_swc_js_minimizer@0.7.2
rspack_plugin_wasm@0.7.2
rspack_plugin_web_worker_template@0.7.2
rspack_plugin_worker@0.7.2
rspack_regex@0.7.2
rspack_storage@0.7.2
rspack_swc_plugin_import@0.7.2
rspack_swc_plugin_ts_collector@0.7.2
rspack_tasks@0.7.2
rspack_tracing@0.7.2
rspack_tracing_perfetto@0.7.2
rspack_util@0.7.2
rspack_watcher@0.7.2
rspack_workspace@0.7.2

Generated by cargo-workspaces

* chore(deps): update dependency @rspack/plugin-react-refresh to ^1.6.0 (#12718)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update dependency @shikijs/transformers to ^3.21.0 (#12719)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update patch crates (#12716)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* test: skip recover-from-error/delete-file in watchCases (#12726)

* feat: support portable cache and disable by default (#12680)

* feat: support portable cache

* feat: disable portable cache by default

* fix: rename reShake to secondaryTreeShaking

* chore: update artificat

* test: use Rstest projects to define tests (#12729)

* test: use rstest projects

* test: update

* test: output.module

* test: update

* chore(deps): update patch npm dependencies (#12717)

* chore(deps): update patch npm dependencies

* fix: downgrade MF

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: neverland <chenjiahan.jait@bytedance.com>

* refactor: move pass into separate folder and driven by run_passes (#12735)

* chore: add rspack_passes.md

* refactor: move pass into separate folder and drive by run_passes

* chore: use function

* refactor: move more into run_passes

* chore: fix stats logging

* chore: fix stats.logging

* refactor(lazy-compilation): use POST request to transfer  ids of active modules (#12678)

* refactor: use post transfer module ids

* test: skip lazy active method

* test: add lazy post request help

* test: add large module id container

* test: add case introduction

* refactor: remove about controller

* test: we are using post method now

* refactor: rename

* fix body parser (vibe-kanban 09797829)

1. 如果其他中间件一个解析好了body,并挂载在req.body 上的花就直直接使用 req.body
2. packages/rspack/src/builtin-plugin/lazy-compilation/middleware.ts body 解析的时候不能简单的使用 string 的拼接,需要考虑多字节符号的截断的问题。

* refactor read module ids from body (vibe-kanban a7874ac1)

packages/rspack/src/builtin-plugin/lazy-compilation/middleware.ts 当req.body 非空时,直接假body中时一组 module ids,如果没有 body 自行拼接完,之后也 parse json string,返回 module ids 数组

* refactor: downgrading to lower web api

* test: ✅ add lazy compilation active cors cases

* chore: update test case doc

* fix: we all need cors header no matter is simple request or not

* refactor:delete cors header setting

* refactor: set cors header should set by user

* Update packages/rspack/hot/lazy-compilation-web.js

Co-authored-by: neverland <chenjiahan.jait@bytedance.com>

* test: fix case name

* chore: api-extract update

* fix: memory leak of requst listeners

* refactor: remove event source handle logic

* fix: jsdom XMLHTTPRequets need strict cors header

* refactor: node lazy compilation client use post too

---------

Co-authored-by: neverland <chenjiahan.jait@bytedance.com>

* refactor: move logger into pass and reorg some passes (#12742)

* refactor: move logger into pass and reorg some passes

* chore: fix bench check

* chore(lint): bump @rslint/core to 0.2.0 and enable default-param-last rule (#12746)

* chore(deps): bump @rslint/core to 0.2.0 and update rslint.json

* fix(lint): enable @typescript-eslint/default-param-last rule

* test: support filter test by absolute path (#12749)

* test: support filter test by absolute path

* Update packages/rspack-test-tools/src/helper/directory.ts

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update packages/rspack-test-tools/src/helper/directory.ts

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update tests/rspack-test/Cache.test.js

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix: lint

* docs: update -t

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* refactor: use newtype for alias artifact (#12754)

* chore: bump `swc_core` from 54 to 55 (#12758)

* chore: bump swc

* chore: bump swc exp

* chore: cargo codegen

* refactor: move all artifacts together (#12755)

* chore: limit threads for codspeed benchmark (#12763)

Limit threads

* chore(deps): update dependency cspell to ^9.6.0 (#12768)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update dependency emnapi to ^1.8.1 (#12769)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update dependency memfs to v4.53.0 (#12770)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* refactor: use compilation readonly ref in codegen optimization hook (#12743)

* refactor: use compilation readonly ref in code generation optimization hook

* chore: debug bench ci

* fix: 🐛 use offical codseed action

* Debug ci

* Remove debugger

* Limit tokio and rayon worker threads

* Revert "Limit tokio and rayon worker threads"

This reverts commit 10fe6ac.

* Update crates/rspack_plugin_progress/src/lib.rs

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: CPunisher <1343316114@qq.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix(browser): update worker format and add e2e test (#12747)

* fix: remove lazyCompilationMiddleware

* chore: change worker entry to iife

* chore: add e2e test

* chore: fix format

* fix: add await for e2e

* fix: remove unused imports

* chore: fix ci yml

* chore: fix ci WASM env injection

* fix: update shared global name

* perf: reuse container plugin

* chore: fix merge conflict

* chore: use official version and update docs

* chore: fix cargo clippy

* chore: bump

* workflow: revert ci

* ci: add CodSpeed performance analysis action

Add GitHub action for continuous benchmarking with CodSpeed. The action supports multiple measurement modes and instruments, including MongoDB instrumentation and caching for faster runs. Includes validation for required inputs and custom runner installation.

* chore: revert cacahe implementation

* chore: use get_referenced_exports

* chore: revert esm depnedency change

* chore: revert esm_import_specifier_dependency change

* refactor(mf): simplify runtime module implementations

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: harpsealjs <lingyucoder@gmail.com>
Co-authored-by: hardfist <yangjianzju@gmail.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Max <fi3ework@gmail.com>
Co-authored-by: Tu Shaokun <53142663+tt-a1i@users.noreply.github.com>
Co-authored-by: neverland <jait.chen@foxmail.com>
Co-authored-by: pshu <stormslowly@gmail.com>
Co-authored-by: jinrui <jerrykingxyz@gmail.com>
Co-authored-by: Gengkun <ahabhgk@gmail.com>
Co-authored-by: CPunisher <1343316114@qq.com>
Co-authored-by: 9aoy <9aoyuao@gmail.com>
Co-authored-by: neverland <chenjiahan.jait@bytedance.com>
Co-authored-by: Cong-Cong Pan <dacongsama@live.com>
Co-authored-by: Agney Menon <agney.menons@live.com>
Co-authored-by: Fy <1114550440@qq.com>
Co-authored-by: senze.fan <151665030@qq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team The issue/pr is created by the member of Rspack.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants